Skip to content

FIX / Fix authentication settings not being saved#24251

Merged
trasher merged 1 commit into
glpi-project:11.0/bugfixesfrom
Mary-Clb:fix/auth-settings-save-button
May 19, 2026
Merged

FIX / Fix authentication settings not being saved#24251
trasher merged 1 commit into
glpi-project:11.0/bugfixesfrom
Mary-Clb:fix/auth-settings-save-button

Conversation

@Mary-Clb
Copy link
Copy Markdown
Contributor

  • I have read the CONTRIBUTING document.
  • I have performed a self-review of my code.
  • I have added tests that prove my fix is effective or that my feature works.
  • This change requires a documentation update.

Description

  • It fixes !44007
  • This PR fixes the authentication settings form submission.

When saving changes from Configuration > Authentication > Setup, the form posts to config.form.php, which only handles this flow when update_auth is present and non-empty. The submit button was sending update_auth without a value, so the request was not processed by the authentication settings update branch. As a result, the request fell back to the generic configuration page and the changes were not saved.
This change adds an explicit value to the update_auth submit button so the authentication settings save flow is correctly triggered.

@Mary-Clb Mary-Clb requested review from Rom1-B and stonebuzz May 19, 2026 09:13
@Mary-Clb Mary-Clb self-assigned this May 19, 2026
@Mary-Clb Mary-Clb added the bug label May 19, 2026
@stonebuzz stonebuzz requested review from cedric-anne and trasher May 19, 2026 10:11
@trasher trasher merged commit e163cee into glpi-project:11.0/bugfixes May 19, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants